Skip to content

Enable auto-merge after repaired review threads#154

Merged
hadamrd merged 1 commit into
trunkfrom
fix/repair-automerge-after-review
May 28, 2026
Merged

Enable auto-merge after repaired review threads#154
hadamrd merged 1 commit into
trunkfrom
fix/repair-automerge-after-review

Conversation

@hadamrd
Copy link
Copy Markdown
Owner

@hadamrd hadamrd commented May 28, 2026

Summary

  • after a repair worker pushes fixes and critic runs, re-check source issue safety and unresolved review threads
  • if review threads are clear, re-enable squash auto-merge on the repaired PR
  • emit explicit repair_automerge_* events so the operator can see why a PR did or did not move

Why

The previous fix made repair workers see and fix inline review threads. This closes the remaining gap: fixed repair PRs should go back onto the merge conveyor instead of sitting open forever.

Tests

  • uv run --extra dev ruff check src/forge_loop/gh.py src/forge_loop/runner/tick.py tests/test_dispatch_axis_filter.py
  • uv run --extra dev pytest tests/test_gh_review_threads.py tests/test_dispatch_axis_filter.py tests/test_worker.py -q
  • git diff --check

@hadamrd hadamrd merged commit 88bf5a4 into trunk May 28, 2026
2 checks passed
@hadamrd hadamrd deleted the fix/repair-automerge-after-review branch May 28, 2026 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant